Skip to content

Remove hardcode names#39

Merged
wine99 merged 1 commit intodev_backend_openvinofrom
remove-hardcode-names
Feb 5, 2026
Merged

Remove hardcode names#39
wine99 merged 1 commit intodev_backend_openvinofrom
remove-hardcode-names

Conversation

@wine99
Copy link
Collaborator

@wine99 wine99 commented Feb 3, 2026

No description provided.

@github-actions github-actions bot added the ggml label Feb 3, 2026
// Put kvcache on device memory for GPU (NPU memory is too small even for kvcache)
if (buffer->usage == GGML_BACKEND_BUFFER_USAGE_ANY && strncmp(tensor->name, "cache_", 6) == 0 && !ctx->is_remote &&
ggml_openvino_get_device_name() == "GPU" && !getenv("GGML_OPENVINO_STATEFUL_EXECUTION")) {
if (strncmp(tensor->name, "cache_", 6) == 0 && !ctx->is_remote && ggml_openvino_get_device_name() == "GPU" &&
Copy link
Collaborator Author

@wine99 wine99 Feb 3, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only place where tensor names appear outside ggml/src/ggml-openvino/openvino.

Inside ggml/src/ggml-openvino/openvino, translate_sessions.cpp still has a few tensor names.

@zhaixuejun1993 @ynimmaga @cavusmustafa

@wine99 wine99 requested a review from cavusmustafa February 3, 2026 09:45
@wine99 wine99 mentioned this pull request Feb 4, 2026
@wine99 wine99 merged commit 750a04a into dev_backend_openvino Feb 5, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant